Skip to content

feat: enable backward.isDefEq.respectTransparency.types by default#13895

Draft
datokrat wants to merge 30 commits into
paul/base/defeq_todo_4_paul_nwmfrom
defeq_todo_4_paul_nwm
Draft

feat: enable backward.isDefEq.respectTransparency.types by default#13895
datokrat wants to merge 30 commits into
paul/base/defeq_todo_4_paul_nwmfrom
defeq_todo_4_paul_nwm

Conversation

@datokrat
Copy link
Copy Markdown
Contributor

No description provided.

@mathlib-lean-pr-testing
Copy link
Copy Markdown

mathlib-lean-pr-testing Bot commented May 29, 2026

Mathlib CI status (docs):

@github-actions github-actions Bot added the toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN label May 29, 2026
@leanprover-bot
Copy link
Copy Markdown
Collaborator

Reference manual CI status:

  • ❗ Reference manual CI can not be attempted yet, as the nightly-testing-2026-05-28 tag does not exist there yet. We will retry when you push more commits. If you rebase your branch onto nightly-with-manual, reference manual CI should run now. You can force reference manual CI using the force-manual-ci label. (2026-05-29 20:15:45)

@github-actions github-actions Bot added the mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN label May 29, 2026
mathlib-nightly-testing Bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request May 29, 2026
@mathlib-lean-pr-testing mathlib-lean-pr-testing Bot added the builds-mathlib CI has verified that Mathlib builds against this PR label May 29, 2026
@datokrat
Copy link
Copy Markdown
Contributor Author

datokrat commented Jun 1, 2026

!radar

@leanprover-radar
Copy link
Copy Markdown

leanprover-radar commented Jun 1, 2026

Benchmark results for bc527c4 against d9ee0ed are in. There are significant results. @datokrat

  • 🟥 build//instructions: +208.0G (+1.84%)

Large changes (1✅, 15🟥)

  • 🟥 build/module/Std.Data.DTreeMap.Internal.Lemmas//instructions: +12.2G (+6.39%)
  • 🟥 build/module/Std.Data.DTreeMap.Lemmas//instructions: +5.3G (+9.29%)
  • 🟥 build/module/Std.Data.DTreeMap.Raw.Lemmas//instructions: +5.1G (+7.49%)
  • build/module/Std.Tactic.BVDecide.Bitblast.BVExpr.Circuit.Lemmas.Operations.Udiv//instructions: -5.3G (-32.57%)
  • 🟥 build/module/Std.Tactic.BVDecide.LRAT.Internal.Formula.RupAddResult//instructions: +7.8G (+19.42%)
  • 🟥 compiled/hashmap//instructions: +1.2G (+36.43%)
  • 🟥 compiled/hashmap//task-clock: +47ms (+26.80%)
  • 🟥 compiled/hashmap//wall-clock: +46ms (+26.70%)
  • 🟥 compiled/liasolver//instructions: +151.5M (+3.79%)
  • 🟥 compiled/parser//instructions: +269.1M (+0.78%)
  • 🟥 compiled/qsort//task-clock: +376ms (+32.09%)
  • 🟥 compiled/qsort//wall-clock: +376ms (+32.09%)
  • 🟥 elab/big_do//instructions: +218.8M (+1.01%)
  • 🟥 elab/big_match_nat_split//instructions: +443.5M (+4.12%)
  • 🟥 elab/big_omega//instructions: +1.1G (+4.97%)
  • 🟥 elab/big_omega_MT//instructions: +1.1G (+5.24%)

Medium changes (1✅, 50🟥)

Too many entries to display here. View the full report on radar instead.

Small changes (16✅, 1200🟥)

Too many entries to display here. View the full report on radar instead.

@leanprover-radar
Copy link
Copy Markdown

leanprover-radar commented Jun 2, 2026

Benchmark results for bdc3f56 against d9ee0ed are in. There are significant results. @implausibility

  • 🟥 build//instructions: +11.0G (+0.10%)

Large changes (1✅, 2🟥)

  • build/module/Std.Tactic.BVDecide.Bitblast.BVExpr.Circuit.Lemmas.Operations.Udiv//instructions: -5.5G (-33.69%)
  • 🟥 build/module/Std.Tactic.BVDecide.LRAT.Internal.Formula.RupAddResult//instructions: +7.1G (+17.82%)
  • 🟥 compiled/parser//instructions: +268.7M (+0.78%)

Medium changes (1✅, 5🟥)

  • 🟥 build/module/Std.Tactic.BVDecide.LRAT.Internal.Clause//instructions: +1.4G (+26.07%) (reduced significance based on absolute threshold)
  • 🟥 build/module/Std.Tactic.BVDecide.LRAT.Internal.Formula.Lemmas//instructions: +2.1G (+13.64%) (reduced significance based on absolute threshold)
  • 🟥 build/module/Std.Tactic.BVDecide.LRAT.Internal.Formula.RatAddResult//instructions: +2.1G (+20.02%) (reduced significance based on absolute threshold)
  • 🟥 build/module/Std.Tactic.BVDecide.LRAT.Internal.Formula.RupAddSound//instructions: +1.1G (+8.22%) (reduced significance based on absolute threshold)
  • 🟥 build/profile/grind//wall-clock: +2s (+45.01%)
  • elab/cbv_decide//instructions: -573.8M (-1.31%)

Small changes (28✅, 36🟥)

Too many entries to display here. View the full report on radar instead.

mathlib-nightly-testing Bot pushed a commit to leanprover-community/batteries that referenced this pull request Jun 2, 2026
mathlib-nightly-testing Bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Jun 2, 2026
@datokrat
Copy link
Copy Markdown
Contributor Author

datokrat commented Jun 2, 2026

!radar

@leanprover-radar
Copy link
Copy Markdown

leanprover-radar commented Jun 2, 2026

Benchmark results for d5f7bfb against d9ee0ed are in. There are significant results. @datokrat

  • build//instructions: -5.3G (-0.05%)

Large changes (1✅, 1🟥)

  • build/module/Std.Tactic.BVDecide.Bitblast.BVExpr.Circuit.Lemmas.Operations.Udiv//instructions: -5.5G (-33.69%)
  • 🟥 compiled/parser//instructions: +269.3M (+0.78%)

Medium changes (1✅)

  • elab/cbv_decide//instructions: -580.6M (-1.32%)

Small changes (30✅, 35🟥)

Too many entries to display here. View the full report on radar instead.

mathlib-nightly-testing Bot pushed a commit to leanprover-community/batteries that referenced this pull request Jun 2, 2026
mathlib-nightly-testing Bot pushed a commit to leanprover-community/mathlib4-nightly-testing that referenced this pull request Jun 2, 2026
kim-em and others added 14 commits June 2, 2026 13:43
This PR refines the "One might expect/hope" annotations so each one states
precisely where the failure surfaces: the stage 2 build, or a specific test
(noting that the `binderNameHint` test failure only appears in a stage 2 build).

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…tions

This PR adds `implicit_reducible` and/or `expose` attributes to definitions
that need to be unfolded during defeq checking without full transparency.
Preparatory commit for removing the TODO workaround in ExprDefEq.lean.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
 (#13584)

This PR fixes some warnings in stage2 due to redundant `@[expose]`.
@datokrat datokrat force-pushed the defeq_todo_4_paul_nwm branch from d5f7bfb to 89bf42e Compare June 2, 2026 12:03
@datokrat datokrat changed the title fix: respectTransparency (new mathlib) feat: enable backward.isDefEq.respectTransparency.types by default Jun 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

builds-mathlib CI has verified that Mathlib builds against this PR force-mathlib-ci mathlib4-nightly-available A branch for this PR exists at leanprover-community/mathlib4-nightly-testing:lean-pr-testing-NNNN toolchain-available A toolchain is available for this PR, at leanprover/lean4-pr-releases:pr-release-NNNN

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants